Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use apt_repository update_cache parameter #1

Merged
merged 1 commit into from Mar 1, 2015

Conversation

barraponto
Copy link
Contributor

Makes for a smaller task :)

@geerlingguy
Copy link
Owner

@barraponto - Since I often run these roles 2, 4, 10 times a day when doing development work (and I generally like having fast playbook runs in general :), I don't like to update apt's cache if it can be avoided... thus I either always have an cache_valid_time of at least 3600 or 86400, or do something like I did in this role where I only update the cache if I know I need to.

At a minimum, I'd like to have cache_valid_time added here.

@barraponto
Copy link
Contributor Author

From the apt_repository update_cache docs:

Run the equivalent of apt-get update when a change occurs. Cache updates are run after making changes.

So, I think it will only run if it needs to.

@geerlingguy
Copy link
Owner

@barraponto - Ah, good eye on that! I never read the docs on apt_repository in that much detail, and was assuming the parameter behaved the same as it does with the general apt module. Learn something new every day!

geerlingguy added a commit that referenced this pull request Mar 1, 2015
Use apt_repository update_cache parameter.
@geerlingguy geerlingguy merged commit f9b5927 into geerlingguy:master Mar 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants